Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[build] Check if patches are applied before applying patches. #13566

Merged
merged 2 commits into from
Feb 6, 2023

Conversation

liushilongbuaa
Copy link
Contributor

@liushilongbuaa liushilongbuaa commented Jan 31, 2023

Why I did it

If make fails, we can't rerun the make process, because existing patches can't apply again.

#13386 missed some change.

How I did it

Check if patches are applied. if yes, don't apply patches again.

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@liushilongbuaa liushilongbuaa marked this pull request as ready for review January 31, 2023 08:04
@liushilongbuaa liushilongbuaa changed the title [build] Check if quilt applied before quilt push [build] Check if patches are applied before applying patches. Jan 31, 2023
@xumia xumia merged commit c7d6b8d into sonic-net:master Feb 6, 2023
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 6, 2023
…net#13566)

Why I did it
If make fails, we can't rerun the make process, because existing patches can't apply again.

sonic-net#13386 missed some change.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #13690

@mssonicbld
Copy link
Collaborator

@liushilongbuaa PR conflicts with 202205 branch

liushilongbuaa added a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Feb 10, 2023
…net#13566)

Why I did it
If make fails, we can't rerun the make process, because existing patches can't apply again.
yxieca pushed a commit that referenced this pull request Feb 10, 2023
#13761)

Why I did it
If make fails, we can't rerun the make process, because existing patches can't apply again.
liushilongbuaa added a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Apr 14, 2023
…net#13566)

Why I did it
If make fails, we can't rerun the make process, because existing patches can't apply again.
@liushilongbuaa
Copy link
Contributor Author

xumia pushed a commit that referenced this pull request Apr 14, 2023
#14662)

Why I did it
If make fails, we can't rerun the make process, because existing patches can't apply again.
#13386 missed some change.

Work item tracking
Microsoft ADO (number only):
How I did it
Check if patches are applied. if yes, don't apply patches again.

How to verify it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants